Skip to content

Rewrite helm client #152

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
git-ival wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Rewrite helm client #152

git-ival wants to merge 7 commits into from

Conversation

@git-ival
Copy link
Contributor

@git-ival git-ival commented Apr 12, 2024 •
edited
Loading

Depends on:

Closes rancher/qa-tasks#1226

Changes:

  • Total rewrite of the helm client utilizing the helm/v3 golang sdk
  • Adds CRUD operations for helm charts and repositories
  • Updates 2 convenience functions for installing certmanager and rancher to utilize the new helm client
  • Adds RESTGetters for helm and kubeconfig

@git-ival git-ival force-pushed the rewrite-helm-client branch 2 times, most recently from c96e32c to c6390dc Compare April 12, 2024 17:13
@git-ival git-ival marked this pull request as ready for review April 12, 2024 19:05
@git-ival git-ival added the automation-enhancement To indicate this is related to enhancements in code related to automation label Apr 12, 2024
@git-ival git-ival force-pushed the rewrite-helm-client branch 3 times, most recently from 444750a to 3307742 Compare April 18, 2024 22:54
@git-ival git-ival force-pushed the rewrite-helm-client branch 2 times, most recently from fd9176c to 8e92be9 Compare April 29, 2024 19:30
@caliskanugur
Copy link
Member

caliskanugur commented Jul 15, 2024

We might need rebases and extra package replaces for this PR

@git-ival git-ival force-pushed the rewrite-helm-client branch from 8e92be9 to d51b709 Compare August 26, 2024 15:48
@git-ival git-ival mentioned this pull request Sep 16, 2024
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@igomez06 igomez06 Awaiting requested review from igomez06 igomez06 is a code owner

@caliskanugur caliskanugur Awaiting requested review from caliskanugur caliskanugur is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

automation-enhancement To indicate this is related to enhancements in code related to automation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[dartboard] Rewrite helm client to use helm v3 go-sdk

2 participants

@git-ival @caliskanugur